Skip to content

Conversation

johncf and others added 10 commits September 19, 2015 21:11
…labnik

Fixes rust-lang#28458

Details about `Arc` may be too soon to be described together with `Sync`... Tell me what you think.

r? @steveklabnik
…steveklabnik

This is a simple grammar fix in which the previous author accidentally repeated a word when (s)he shouldn't have.
This reduces some clones of `Vec`s. These are not deep copies since the
token tree is made using `Rc`s, so this won't be a major improvement.

r? @eddyb
@rust-highfive
Copy link
Contributor

r? @nrc

(rust_highfive has picked a reviewer for you, use r? to override)

@Manishearth
Copy link
Member Author

@bors r+ p=20

@bors
Copy link
Collaborator

bors commented Sep 20, 2015

📌 Commit d7ec69a has been approved by Manishearth

@Manishearth Manishearth assigned Manishearth and unassigned nrc Sep 20, 2015
bors added a commit that referenced this pull request Sep 20, 2015
@bors
Copy link
Collaborator

bors commented Sep 20, 2015

⌛ Testing commit d7ec69a with merge 25aaeb4...

@bors bors merged commit d7ec69a into rust-lang:master Sep 20, 2015
@Centril Centril added the rollup A PR which is a rollup label Oct 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rollup A PR which is a rollup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants